* Calling $user->isActiveUser() looks a bit redundant. :) Consider either shortening it to isActive() or changing the last word to be clearer, say $user->isActiveEditor()
* $wgActiveUserEditcount <- consider fully casing here; "editcount" isn't a word. :D -> $wgActiveUserEditCount
* $oldTime is dumped into SQL without any escaping or quoting. This will happen to work on MySQL but will fail on PostgreSQL, and is generally a bad practice. Use addQuotes() to ensure the formatted timestamp string is escaped and quoted for your raw SQL construct
* the database result object is not freed, which will leak resources if used in a long-running script
(default 100) with the new subpage-move functionality of Special:Movepage
* Hooks display in Special:Version is now disabled by default, use
$wgSpecialVersionShowHooks = true; to enable it.
-* $wgActiveUserEditcount sets the number of edits that must be performed over
- a certain number of days to be considered active
-* $wgActiveUserDays is that number of days
=== New features in 1.13 ===
* (bug 14263) Show a diff of the revert on rollback notification page.
* (bug 13434) Show a warning when hash identical files exist
* Sidebar is now cached for all languages
-* The User class now contains a public function called isActiveUser. Figures
- out if a user is active based on at least $wgActiveUserEditcount number of
- edits in the last $wgActiveUserDays days.
=== Bug fixes in 1.13 ===
*/
$wgDeleteRevisionsLimit = 0;
-/**
- * Used to figure out if a user is "active" or not. User::isActiveUser()
- * sees if a user has made at least $wgActiveUserEditcount number of edits
- * within the last $wgActiveUserDays days.
- */
-$wgActiveUserEditcount = 30;
-$wgActiveUserDays = 30;
-
# Proxy scanner settings
#
function isNewbie() {
return !$this->isAllowed( 'autoconfirmed' );
}
-
- /**
- * Is the user active? We check to see if they've made at least
- * X number of edits in the last Y days.
- *
- * @return bool true if the user is active, false if not
- */
- public function isActiveUser() {
- global $wgActiveUserEditcount, $wgActiveUserDays;
- $dbr = wfGetDB( DB_SLAVE );
-
- // Stolen without shame from RC
- $cutoff_unixtime = time() - ( $wgActiveUserDays * 86400 );
- $cutoff_unixtime = $cutoff_unixtime - ( $cutoff_unixtime % 86400 );
- $oldTime = $dbr->timestamp( $cutoff_unixtime );
-
- $res = $dbr->select( 'revision', '1',
- array( 'rev_user_text' => $this->getName(), "rev_timestamp > $oldTime"),
- __METHOD__,
- array('LIMIT' => $wgActiveUserEditcount ) );
-
- return $dbr->numRows($res) == $wgActiveUserEditcount ? true : false;
- }
/**
* Check to see if the given clear-text password is one of the accepted passwords